-
Notifications
You must be signed in to change notification settings - Fork 4.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update metrics in traefik-kubernetes.json grafana dashboard #10591
base: v3.0
Are you sure you want to change the base?
Update metrics in traefik-kubernetes.json grafana dashboard #10591
Conversation
Hello, I ran into this issue also and I'd like to see a PR for this merged. Looking over this PR, there are a couple improvements:
Here is a commit of this MR plus these additional changes: @Sad-Soul-Eater Perhaps you can incorporate these in this PR? |
Thank you @Sad-Soul-Eater! This PR LGTM. ping @jspdown @mmatur when you have a chance can you take a look please. |
Gentle ping, thanks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What does this PR do?
Fix variables:
entrypoint
service
Fix panels:
Requests per Entrypoint
Apdex score
Connections per Entrypoint
Motivation
In v3.0 were metrics changes, which broke some of the dashboard panels and variables.
Fixes #10691
Additional Notes
Connections per Service
panel remains broken because it relies on the oldtraefik_service_open_connections
metric, but I haven't found any other metrics that can be used to replace the old one.